-
Notifications
You must be signed in to change notification settings - Fork 119
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix bug when declaring a cmap and color argument would produce len(df… #200
Conversation
@gidden, is this good to go? can you do a quick rebase, please? |
should be good to go for a review @danielhuppmann |
I will rebase this once #213 is in with additional testing/updates. |
…) colors instead of len(lines)
@danielhuppmann I rebased this now for most recent CI updates. Should be good to go. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @gidden for this bugfix!
…) colors instead of len(lines)
Please confirm that this PR has done the following:
Adding to RELEASE_NOTES.md (remove section after adding to RELEASE_NOTES.md)
Please add a single line in the release notes similar to the following:
Description of PR
Please describe the changes introduced by this PR.